feat(telemetry): Add telemetry on signals INTELLIJ-180 #119

kmruiz · 2025-01-16T18:51:38Z

Description

Checklist

New tests and/or benchmarks are included.
Documentation is changed or added.
Changelog is updated accordingly.
I have signed the MongoDB Contributor License Agreement (https://www.mongodb.com/legal/contributor-agreement).

Open Questions

github-actions · 2025-01-16T18:53:48Z

Coverage Report

Overall Project	79.39% `-0.38%`	✅
Files changed	90.21%	✅

File	Coverage
FieldCheckInspectionBridge.kt	99.49%	✅
TelemetryEvent.kt	96.08%	✅
InspectionStatusChangedProbe.kt	94.39% `-5.61%`	✅
DataGripMongoDbDriver.kt	94.06% `-1.88%`	🚫
AbstractMongoDbInspectionBridge.kt	92.22% `-5.56%`	🚫
IndexCheckInspectionBridge.kt	86.89%	✅
NamespaceCheckInspectionBridge.kt	84.93% `-7.12%`	✅
SpringAtQueryDialectParser.kt	30.4% `-3.29%`	🚫

…spections properly

github-actions · 2025-01-17T15:33:44Z

🤖 Benchmark Comparison for chore: fail safe in case telemetry fails

Benchmark	Previous	Current	Change
`com.mongodb.jbplugin.jmh.SampleBenchmark.init`	`2,600,565,477.62 ops/s`	`2,664,140,530.48 ops/s`	`2.44%`

github-actions · 2025-01-17T17:00:01Z

🤖 Benchmark Comparison for Merge branch 'main' into feat/INTELLIJ-180

Benchmark	Previous	Current	Change
`com.mongodb.jbplugin.jmh.SampleBenchmark.init`	`2,668,742,126.31 ops/s`	`2,602,281,813.98 ops/s`	`-2.49%`

himanshusinghs

amazing work 🚀

himanshusinghs · 2025-01-17T17:09:51Z

...src/main/kotlin/com/mongodb/jbplugin/accessadapter/datagrip/adapter/DataGripMongoDbDriver.kt

@@ -206,8 +210,16 @@ internal class DataGripMongoDbDriver(

            withTimeout(timeout) {
                val listOfResults = mutableListOf<T>()
-                val resultSet = statement.executeQuery() ?: return@withTimeout emptyList()
+                val queryResult = runCatching { statement.executeQuery() }


nice catch 👁️

kmruiz added 2 commits January 16, 2025 19:22

chore: tmp

6f3d104

chore: add error message when querying mongodb fails

cb7b558

kmruiz self-assigned this Jan 16, 2025

github-actions bot added the feat label Jan 16, 2025